Skip to content

ForEach: add default value for Elements parameter#9099

Open
qoomak wants to merge 1 commit into
stagingfrom
for-each-default-value
Open

ForEach: add default value for Elements parameter#9099
qoomak wants to merge 1 commit into
stagingfrom
for-each-default-value

Conversation

@qoomak

@qoomak qoomak commented Mar 22, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds @DefaultValue({"item1", "item2"}) to the Elements parameter in both Flink and Lite ForEachTransformer
  • Prevents an immediate mandatory-field validation error when a For Each node is added to the graph before the user has had a chance to fill in the field

Test plan

  • Add a For Each node to a scenario — verify no validation error appears immediately
  • Verify the Elements field is pre-filled with {"item1", "item2"}
  • Verify that changing Elements to a real expression works correctly

@JulianWielga JulianWielga requested a review from mslabek March 24, 2026 14:59
@qoomak qoomak force-pushed the for-each-default-value branch 2 times, most recently from fedf3b3 to bce4642 Compare March 25, 2026 11:10
Prevents immediate validation error when node is added before user has a chance to fill in the field.
@piotrp piotrp force-pushed the for-each-default-value branch from bce4642 to 1275c04 Compare March 26, 2026 21:37
@github-actions

Copy link
Copy Markdown
Contributor

🟢 Cypress snapshots are up to date. No changes needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants